fix: Update Display test results to .res file - #506
Conversation
Fix Display testscripts to update the PASS/FAIL status to result file Signed-off-by: Vamsee Narapareddi <vnarapar@qti.qualcomm.com>
Srikanth Muppandam (smuppand)
left a comment
There was a problem hiding this comment.
This PASS write duplicates the existing final result write at line 279, and the FAIL path duplicates it again at line 288 across the files it is same. So i would suggest it is a duplicate result updation.
|
|
||
| if [ "$final" = "PASS" ]; then | ||
| log_pass "${TESTNAME} : PASS" | ||
| echo "${TESTNAME} PASS" >"$RES_FILE" |
There was a problem hiding this comment.
This PASS write duplicates the existing final result write at line 279, and the FAIL path duplicates it again at line 288.
There was a problem hiding this comment.
This PASS write duplicates the existing final result write at line 279, and the FAIL path duplicates it again at line 288.
vnarapar Can you relook?
There was a problem hiding this comment.
Sry my bad. i will check and update
|
@ualcomm/qualcomm-linux-testing.triage This pull request has been marked as stale due to 30 days of inactivity and will automatically close after an additional 5 days. |
Fix Display testscripts to update the PASS/FAIL status to result file